Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set depthMode for 2d custom layers #7432

Merged
merged 4 commits into from
Oct 17, 2018
Merged

Conversation

markusjohnsson
Copy link
Contributor

Launch Checklist

  • briefly describe the changes in this PR
  • write tests for all new functionality
  • manually test the debug page

This PR fixes #7422 by setting depthMode in draw_custom.js for 2d layers.

@mourner mourner requested a review from ansis October 17, 2018 09:33
Copy link
Contributor

@ansis ansis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@markusjohnsson thanks for tracking this down, adding the test and opening the pr! It looks good. I'm only seeing that one small ReadWrite line that needs to be changed and then this should be good to be merged.

Thanks!

src/render/draw_custom.js Outdated Show resolved Hide resolved
@markusjohnsson
Copy link
Contributor Author

@ansis thanks, I've applied the changes

Copy link
Contributor

@ansis ansis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ansis ansis merged commit 71ed65a into mapbox:master Oct 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Depth/layer order in 2d custom layers
2 participants